Skip to content

feat: support bunx for js configs#305

Closed
nwidynski wants to merge 1 commit into
JamieMason:mainfrom
nwidynski:patch-1
Closed

feat: support bunx for js configs#305
nwidynski wants to merge 1 commit into
JamieMason:mainfrom
nwidynski:patch-1

Conversation

@nwidynski
Copy link
Copy Markdown
Contributor

@nwidynski nwidynski commented Oct 11, 2025

Description (What)

This change detects bun and when present uses bunx instead of npx for executing js-based config files.

Justification (Why)

Bun package installations may not be compatible with the npm task runner. This is the case when using a custom BUN_INSTALL_CACHE_DIR with the isolated linker setting.

How Can This Be Tested?

  1. Setup a new bun repository
  2. Configure a custom cache.dir and "isolated" linker setting via bunfig.toml
  3. Add syncpack and a syncpack.config.ts
  4. Attempt to run syncpack lint

Result: ✗ Node.js/npx/tsx process failed with stderr: sh: tsx: command not found

@nwidynski nwidynski changed the title feat: support bun environments for js configs feat: support bunx for js configs Oct 11, 2025
@JamieMason
Copy link
Copy Markdown
Owner

Thanks @nwidynski 👍

@nwidynski
Copy link
Copy Markdown
Contributor Author

@JamieMason Any chance to get this merged?

JamieMason pushed a commit that referenced this pull request Nov 2, 2025
@JamieMason
Copy link
Copy Markdown
Owner

It's in the v14-alpha dev branch, I want to test a big refactor some more before I next cut a release.

@JamieMason JamieMason closed this in 98c0bf3 Nov 4, 2025
@JamieMason
Copy link
Copy Markdown
Owner

Released in 14.0.0-alpha.24

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants